-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
增加了对B站8K超高清视频的支持 #978
增加了对B站8K超高清视频的支持 #978
Conversation
我意识到我犯了一个错误,我在修改代码之前忘记先获取上游的修改了,请问我现在应该如何解决 |
你先 rebase 一下你的分支,再把 1269714 这个 commit cherry-pick 过去 |
Co-authored-by: Xinzhao Xu <z2d@jifangcheng.com>
Co-authored-by: Xinzhao Xu <z2d@jifangcheng.com>
Co-authored-by: Xinzhao Xu <z2d@jifangcheng.com>
非常抱歉,给您添麻烦了,我现在已经解决完我的分支了,您可以开始检查本次提交了 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
目前B站只在电视端和Safari浏览器支持8k视频,所以设置了了一个UA,那个地方如果后续B站进一步开放了8k的设备,可以去掉。
此外,无关本次修改的代码,我还有一个小问题,是我的操作哪里存在问题吗?我的每一次拉取请求都会带上从我第一次贡献代码开始的所有提交,尽管之前那些提交已经被合并
close #977